cdc4075ddbc642f3a82b8c2a0e6120636e2cbd0f,xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-handlers/xwiki-platform-extension-handler-xar/src/main/java/org/xwiki/extension/xar/internal/handler/XarExtensionHandler.java,XarExtensionHandler,upgrade,#LocalExtension#LocalExtension#String#,69

Before Change



        XarLocalExtension previousXarExtension;
        try {
            previousXarExtension = (XarLocalExtension) this.xarRepository.resolve(previousLocalExtension.getId());
        } catch (ResolveException e) {
            // Not supposed to be possible
            throw new InstallException("Failed to get xar extension [" + previousLocalExtension.getId()
                + "] from xar repository", e);
        }

        // Install new pages
        install(previousXarExtension, newLocalExtension, namespace);

        // Uninstall old version pages not anymore in the new version
        Set<XarEntry> previousPages = new HashSet<XarEntry>(previousXarExtension.getPages());

        List<XarEntry> newPages;
        try {

After Change


        // Produce a pages diff between previous and new version
        Set<XarEntry> previousPages = new HashSet<XarEntry>();
        try {
            XarLocalExtension previousXarExtension =
                (XarLocalExtension) this.xarRepository.resolve(previousLocalExtension.getId());
            previousPages.addAll(previousXarExtension.getPages());
        } catch (ResolveException e) {
            // Not supposed to be possible
            throw new InstallException("Failed to get xar extension [" + previousLocalExtension.getId()